Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DPB] [Mellanox] added capability files for SN4700 platform #6278

Merged

Conversation

vadymhlushko-mlnx
Copy link
Contributor

@vadymhlushko-mlnx vadymhlushko-mlnx commented Dec 23, 2020

Signed-off-by: Vadym Hlushko vadymh@nvidia.com

- Why I did it

platform.json and hwsku.json files has not a full set of speeds for split modes

- How I did it

Extended set of speeds for split modes for SN4700 platform

- How to verify it

Manually run DPB CLI commands

- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Vadym Hlushko <vadymh@nvidia.com>
@vadymhlushko-mlnx
Copy link
Contributor Author

retest this please

@vadymhlushko-mlnx
Copy link
Contributor Author

Please don't merge this PR until [DPB][MLNX][YANG-models] extended regex pattern according to MLNX systems speeds requirements PR will be merged

@liat-grozovik
Copy link
Collaborator

retest vs please

@liat-grozovik
Copy link
Collaborator

retest vsimage please

@liat-grozovik liat-grozovik changed the title [DPB][SN4700] extended set of speeds for split modes [DPB] [Mellanox] added capability files for SN4700 platform Jan 13, 2021
@liat-grozovik
Copy link
Collaborator

retest vsimage please

@liat-grozovik
Copy link
Collaborator

@lguohan can you please merge it? the azure build is stuck for few days

@lguohan lguohan merged commit 709c1ec into sonic-net:master Jan 25, 2021
@vadymhlushko-mlnx vadymhlushko-mlnx deleted the sn4700_dpb_capability_speeds_fix branch January 25, 2021 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants